Skip to content

edit Media Type Object#1171

Merged
webron merged 2 commits intoOAI:OpenAPI.nextfrom
krishahn:media-type-object
Jun 13, 2017
Merged

edit Media Type Object#1171
webron merged 2 commits intoOAI:OpenAPI.nextfrom
krishahn:media-type-object

Conversation

@krishahn
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread versions/3.0.md Outdated
<a name="mediaTypeEncoding"></a>encoding | Map[`string`, [Encoding Object](#encodingObject)] | A map between a property name and its encoding information. The key, being the property name, MUST exist in the schema as a property. The encoding object SHOULD only apply to `requestBody` objects when the content type is `multipart`.
<a name="mediaTypeExample"></a>example | Any | Example of the media type. The example object SHOULD be in the correct format as specified by the media type. The `example` object is mutually exclusive of the `examples` object. Furthermore, if referencing a `schema` which contains an example, the `example` value SHALL _override_ the example provided by the schema.
<a name="mediaTypeExamples"></a>examples | Map[ `string`, [Example Object](#exampleObject) \| [Reference Object](#referenceObject)] | Examples of the media type. Each example object SHOULD match the media type and specified schema if present. The `examples` object is mutually exclusive of the `example` object. Furthermore, if referencing a `schema` which contains an example, the `examples` value SHALL _override_ the example provided by the schema.
<a name="mediaTypeEncoding"></a>encoding | Map[`string`, [Encoding Object](#encodingObject)] | A map between a property name and its encoding information. The key, being the property name, MUST exist in the schema as a property. The encoding object SHOULD apply only to `requestBody` objects when the content type is `multipart`.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not splitting infinitives may may make the text easier to read for those whose first language is not English. So prefer 'only apply to' over 'apply only to'.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tks, i'll stop nit-picking. I thought the adverb had to appear between the preposition and a verb to split it. Regarding the placement of only, this isn't the first time somebody complained about my moving "only" as far to the end of the sentence as possible. I think doing so makes things sound a little British--not good. On the plus side, doing so often prevents embarrassing wording mistakes.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Disclosure: British. :)

Copy link
Copy Markdown
Contributor Author

@krishahn krishahn Jun 11, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops. fixing only placement per your comment.

Copy link
Copy Markdown
Member

@webron webron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved by @emckean.

@webron webron merged commit 283452e into OAI:OpenAPI.next Jun 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants